-
Notifications
You must be signed in to change notification settings - Fork 346
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[actions] add workflow file for containerd integration testing #968
[actions] add workflow file for containerd integration testing #968
Conversation
df155ad
to
e264e69
Compare
Hi, @guni1192 👋 We have great news! We have passed the contained integration test!!! So this PR has become more important for us. Feel free to let me know if you don't have enough time to complete this PR ;) |
@utam0k Excellent!!! I'm so sorry I've been out of touch! |
@guni1192 Sure! I'm glad to hear that. But please keep in mind, if you need to overdo it to finish it, feel free to let me know. It is not important that it be done by the end of this month. What is most important is that you have fun developing this ;) |
e264e69
to
904aa8f
Compare
I appreciate your concern.
After the above actions, two tests have failed and I will investigate the cause.
|
Hey, I re-tried the two cases on my machine through vagrant several times, and TestRegressionIssue6429 always passed, and TestContainerNoBinaryExists seems to fail. I'm trying to investigate that. |
@YJDoc2 Yes. I will do it today! |
Signed-off-by: Takashi IIGUNI <iiguni.tks@gmail.com>
904aa8f
to
cb718ae
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@guni1192 Awesome!!! |
related: #531